Skip to content

Conversation

@nordic-krch
Copy link
Contributor

Update hal_nordic with errata workaround in nrfx_spim.

Upstream PR #: 89489

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 5, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_nordic zephyrproject-rtos/hal_nordic@8cdcc33 zephyrproject-rtos/hal_nordic@a5a2277 (master) zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@nordic-krch nordic-krch force-pushed the ncs/spim_nrf54lm20 branch from d6c9883 to 77ab32a Compare May 7, 2025 12:53
@NordicBuilder NordicBuilder removed the DNM label May 7, 2025
So that we have access to the new nrf_gpio_pin_drive_get() API.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
(cherry picked from commit becf5ed)
@nordic-krch nordic-krch force-pushed the ncs/spim_nrf54lm20 branch from 77ab32a to 16710b9 Compare May 8, 2025 14:30
west.yml Outdated
- hal
- name: hal_nordic
revision: 119ff5b5ec7a413ca2f64ae1928c79eee1e7b7b2
revision: 37c38a471c042d890180cfb79cf788a32a006b80
Copy link
Contributor

@jfischer-no jfischer-no May 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upstream PR #: 89489 is outdated and will never be merged. Also why do we need two commits to changed the same version? Can you cherry-pick d463ea1 from the upmerge PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need 2 commits because last commit will not apply without conflicts. Cherry-picked newer commit.

@nordic-krch nordic-krch force-pushed the ncs/spim_nrf54lm20 branch from 16710b9 to 3ef16b5 Compare May 9, 2025 08:20
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 9, 2025

@jfischer-no jfischer-no merged commit 1290794 into nrfconnect:main May 9, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants